Skip to content

Enable agent-browse without API key using Claude Code subscription #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
shrey150 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Enable agent-browse without API key using Claude Code subscription #21

shrey150 wants to merge 1 commit into from

Conversation

@shrey150
Copy link

@shrey150 shrey150 commented Dec 12, 2025 •
edited
Loading

What changed

  • Add getClaudeCodeApiKey() to retrieve token from system keychain
  • Support macOS Keychain, Windows Credential Manager, Linux secret storage
  • Fall back to ANTHROPIC_API_KEY env var if keychain not available
  • Update error messages with clear setup instructions

Testing strategy

Note: only tested on macOS

  • use env -i to create a new environment
  • test that it pulls from system keychain
  • if not in system keychain, error is shown
  • running claude setup-token then loads API key

Kylejeong2
Kylejeong2 approved these changes Dec 13, 2025
View reviewed changes
Copy link
Member

@Kylejeong2 Kylejeong2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm,

only nit is are we going to use emojis in the console errors? seems very AI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pkiv pkiv pkiv approved these changes

@Kylejeong2 Kylejeong2 Kylejeong2 approved these changes

@miguelg719 miguelg719 Awaiting requested review from miguelg719

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shrey150 @pkiv @Kylejeong2